Skip to content

test: updated flow for GitSync#34239

Merged
yatinappsmith merged 10 commits intoreleasefrom
test/fixFlakyGitSync2
Jun 28, 2024
Merged

test: updated flow for GitSync#34239
yatinappsmith merged 10 commits intoreleasefrom
test/fixFlakyGitSync2

Conversation

@NandanAnantharamu
Copy link
Contributor

@NandanAnantharamu NandanAnantharamu commented Jun 14, 2024

RCA:
Click on the dropdown was not working intermittently

Solution:
updated the flow,

  1. Wait for the destination dropdown
  2. Click on the destination dropdown
  3. Assert the branch name visible post dropdown has opened
  4. Click on the destination branch

Summary by CodeRabbit

  • Bug Fixes

    • Enhanced the mergeBranch process in GitSync to ensure better stability and accuracy by adding assertions for element appearances, disappearances, network status, and the presence of a destination branch.
  • Tests

    • Switched limited test spec from Fork_Template_spec.js to GitSyncedApps_spec.js and added GitBugs_Spec.ts for improved test coverage and accuracy.
  • Refactor

    • Changed className attribute to data-testid in the Select component within the GitSync merge tab for improved testing and readability.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Adding this label to a PR prevents it from being listed in the changelog Test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants